-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Uncrispyfy NAV's custom crispy templates #3008
Conversation
Used extensively in /alertprofiles/matchfields/ and /threshold/add
Used somewhere in /arnold/ (could not find where exactly though) and in "'My stuff' quick links" under /preferences/ if you have saved any quick links
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
Test results 9 files 9 suites 8m 35s ⏱️ Results for commit 385d112. ♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3008 +/- ##
==========================================
- Coverage 56.76% 56.60% -0.16%
==========================================
Files 603 603
Lines 43720 43727 +7
Branches 48 48
==========================================
- Hits 24816 24752 -64
- Misses 18892 18963 +71
Partials 12 12 ☔ View full report in Codecov by Sentry. |
I took the liberty of changing the base to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok, just a tiny nitpick :)
Introduces reusable templates to #2794:
horizontal_checkbox.html
for uncrispyfyingCheckBox()
field_helptext_as_icon.html
for uncrispyfyingHelpField()
Depends on #3000. So only last two commits are original, the rest will be rebased once #3000 is merged